Skip to content

試験的機能からETAアシストを廃止しリモート設定のみで自動有効化するよう変更#6413

Merged
TinyKitten merged 1 commit into
devfrom
claude/eta-assist-auto-enable-nenfjv
Jul 13, 2026
Merged

試験的機能からETAアシストを廃止しリモート設定のみで自動有効化するよう変更#6413
TinyKitten merged 1 commit into
devfrom
claude/eta-assist-auto-enable-nenfjv

Conversation

@TinyKitten

@TinyKitten TinyKitten commented Jul 13, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Member

概要

リモート設定(eta_assist_enabled)が許可している場合に ETA アシスト（到着判定の改善）を自動的に有効化するよう変更し、試験的機能設定から手動トグル項目を削除しました。従来はリモート設定のマスタースイッチに加えてユーザーの手動トグルが必要でしたが、手動トグルを廃止し、リモート設定のみで判定します。リモート設定はサーバー側キルスイッチとして継続して残します。

変更の種類

  • バグ修正
  • 新機能
  • リファクタリング
  • ドキュメント
  • CI/CD
  • その他

変更内容

  • 試験的機能設定画面(ExperimentalSettings)から ETA アシストのトグル項目・説明文・トグル処理を削除
  • isEtaAssistEnabled() を「リモート設定 AND 手動トグル」から「リモート設定(eta_assist_enabled)のみ」に変更し、統合により不要となった isEtaAssistRemoteEnabled() を削除
  • 手動トグル用の atom(etaAssistManualEnabledAtom)を削除
  • 起動時の手動トグル復元処理(Permitted)と永続化ストレージキー(ETA_ASSIST_MANUAL_ENABLED)を削除
  • 未使用となった翻訳文言(etaAssistTitle / etaAssistDescription、ja/en)を削除
  • 関連テスト(remoteConfig.test.ts / ExperimentalSettings.test.tsx)を新仕様へ更新

テスト

  • npm run lint が通ること
  • npm test が通ること
  • npm run typecheck が通ること

関連Issue

スクリーンショット（任意）

Generated by Claude Code

Summary by CodeRabbit

  • 変更

    • 設定画面からETA補助の手動切り替え項目を削除しました。
    • ポートレートモードの設定項目を追加し、縦持ち時のレイアウトを切り替えられるようにしました。
    • テレメトリに関する説明文を更新し、端末の地理座標が分析サーバーへ送信されることを明示しました。
    • 設定値の保存・復元対象を更新しました。

  • テスト

    • ETA補助がリモート設定に基づいてのみ有効になる動作を確認するテストを更新しました。

@coderabbitai

coderabbitai Bot commented Jul 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 354d8d4d-6759-48f8-bbd8-f47a2cc62d7c

📥 Commits

Reviewing files that changed from the base of the PR and between 6bfde55 and 2a919e7.

📒 Files selected for processing (9)
  • assets/translations/en.json
  • assets/translations/ja.json
  • src/components/Permitted.tsx
  • src/constants/storage.ts
  • src/lib/remoteConfig.test.ts
  • src/lib/remoteConfig.ts
  • src/screens/ExperimentalSettings.test.tsx
  • src/screens/ExperimentalSettings.tsx
  • src/store/atoms/experimental.ts
💤 Files with no reviewable changes (4)
  • src/constants/storage.ts
  • assets/translations/ja.json
  • assets/translations/en.json
  • src/store/atoms/experimental.ts
📝 Walkthrough

Walkthrough

ETA補助をRemote Configのマスタースイッチのみで判定するよう変更し、手動トグル、関連する画面表示・保存復元処理・翻訳を削除しました。ポートレートモードなどの設定キーと翻訳を追加しています。

Changes

ETA補助設定の整理

Layer / File(s) Summary
Remote Config単独判定
src/lib/remoteConfig.ts, src/lib/remoteConfig.test.ts		 ETA補助の有効判定から手動トグルとの合成を削除し、Remote Configの値またはfalseを返す仕様へ変更しました。
実験設定画面のETA補助削除
src/screens/ExperimentalSettings.tsx, src/screens/ExperimentalSettings.test.tsx, assets/translations/*.json		 実験設定画面からETA補助手動トグルを削除し、ポートレートモードとテレメトリに関する表示・テスト・翻訳を更新しました。
設定キーと復元処理の整理
src/constants/storage.ts, src/components/Permitted.tsx, src/store/atoms/experimental.ts		 ETA補助手動設定のストレージキーと復元処理を削除し、ポートレートモードなどの設定キーを追加しました。

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

Suggested labels: react

Poem

ぴょんと跳ねたら、補助は消え
遠隔スイッチが道を決め
縦持ち画面が顔を出す
設定の鍵も軽くなり
月夜のうさぎは、テストを祝う ─ 🐇

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PRの主変更であるETAアシストの廃止とリモート設定のみでの自動有効化を的確に表しています。
Description check ✅ Passed テンプレートの必須項目である概要、変更の種類、変更内容、テストが揃っており、内容も十分具体的です。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/eta-assist-auto-enable-nenfjv

Comment @coderabbitai help to get the list of available commands.

@TinyKitten TinyKitten self-assigned this Jul 13, 2026
@github-actions github-actions Bot added the react label Jul 13, 2026
@TinyKitten

TinyKitten commented Jul 13, 2026

Copy link
Copy Markdown
Member Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jul 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@TinyKitten TinyKitten merged commit 3bf50be into dev Jul 13, 2026
7 checks passed
@TinyKitten TinyKitten deleted the claude/eta-assist-auto-enable-nenfjv branch July 13, 2026 10:25
@TinyKitten TinyKitten mentioned this pull request Jul 13, 2026
Merged
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

@TinyKitten TinyKitten

Labels

react

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@TinyKitten @claude